Skip to content

Conversation

lcnr
Copy link
Contributor

@lcnr lcnr commented Feb 23, 2022

probably seems worth explicitly mentioning this, considering that for myself, that's actually the feature i care more about ^^

@rust-highfive
Copy link
Contributor

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive
Copy link
Contributor

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against stable. Please double check that you specified the right target!

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 23, 2022
@lcnr
Copy link
Contributor Author

lcnr commented Feb 23, 2022

idk if changing this directly on stable is correct

@Mark-Simulacrum
Copy link
Member

No, we don't typically merge to stable unless we're actually meaningfully changing something (since that takes a full bors build cycle).

If you can refile against master (and "back"port the release notes from stable), that would be good. There's a few issues noted in #94278 that might also be nice to patch up on the way.


Unrelated to the contents of this PR, you probably don't want to be creating branches in rust-lang/rust (e.g., lcnr-patch-1). The GitHub UI unfortunately forces this if you edit on the frontend and have write access, but those branches are visible to everyone and are somewhat noisy as a result.

@lcnr lcnr closed this Feb 23, 2022
@lcnr lcnr deleted the lcnr-patch-1 branch February 23, 2022 15:08
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Feb 25, 2022
update const_generics_defaults release notes

supersedes rust-lang#94294

r? `@Mark-Simulacrum`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants